Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow for Jira in title #10

Merged
merged 2 commits into from
Mar 11, 2024
Merged

Allow for Jira in title #10

merged 2 commits into from
Mar 11, 2024

Conversation

frankywahl
Copy link
Member

Allow for Jira ticket in title

Since a PR is opened, it cannot have it's base branch edited. That means
that if we forget to put a PR number, we have to close it and open it
again.

With this change, it is now possible to put the ticket in the PR title
to be picked up as well and linked properly.

Since a PR is opened, it cannot have it's base branch edited. That means
that if we forget to put a PR number, we have to close it and open it
again.

With this change, it is now possible to put the ticket in the PR title
to be picked up as well and linked properly.
Copy link
Contributor

@pmrotule pmrotule left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought it was a requirement to have the branch name starting with the Jira ticket number in order to automatically create the link to the pull request in Jira, but according to the documentation, it will also detects it in the PR title.

I was aware of the limitation from using the branch name like you mentioned in your PR description and we tried to prevent it by adding a pre-commit hook. That being said, your solution is much better! No pre-commit hook needed 🙌

@frankywahl frankywahl merged commit b94c33c into main Mar 11, 2024
2 checks passed
@frankywahl frankywahl deleted the allow-for-jira-in-title branch March 11, 2024 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants